Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New landing #362

Merged
merged 40 commits into from
Feb 13, 2024
Merged

New landing #362

merged 40 commits into from
Feb 13, 2024

Conversation

Ledchig
Copy link
Contributor

@Ledchig Ledchig commented Aug 24, 2023

Сделал пока только под мобильные устройства

@Ledchig Ledchig marked this pull request as ready for review August 25, 2023 12:10
@dzencot
Copy link
Collaborator

dzencot commented Sep 3, 2023

А можешь плз задеплоить, чтоб тестировщики глянули?

@Ledchig
Copy link
Contributor Author

Ledchig commented Sep 3, 2023

А можешь плз задеплоить, чтоб тестировщики глянули?

Задеплоил. Вот ссылка https://ledchig-runit.onrender.com/

Я ещё в процессе. Нужно фоновые картинки добавить, темы не настроены и более мелких исправлений тоже хватает)

@Ledchig
Copy link
Contributor Author

Ledchig commented Sep 3, 2023

А так. Возможно, стоит подольше ждать при попытке перехода по ссылке, т.к. рендер автоматом деплоит при попытке коннекта и отрубает когда ни кого нет на сайте для экономии.

@dzencot
Copy link
Collaborator

dzencot commented Sep 4, 2023

А, если в процессе то ладно. Тогда нужно пр в драфт перенести.

@fey fey marked this pull request as draft September 6, 2023 09:01
@fey fey linked an issue Sep 8, 2023 that may be closed by this pull request
@faciledictu
Copy link
Contributor

Привет! Как дела с лендингом? Может, помочь?

@Ledchig
Copy link
Contributor Author

Ledchig commented Jan 9, 2024

Не понял чуток. Все как и должно быть или они черные на темной теме?

@Shvidunov
Copy link

Все поправил. Скроллбар таким и задумывался вроде как. Можно конечно убрать скроллбар, но тогда совсем не интуитивно будет в плане использования. Может и просто карусель с кнопками сделать лучше, т.к. она понятна и привычна пользователям.

мне кажется, что лучше карусель ) скроллбар как будто случайно туда попал. Может как вариант его вид поменять - какой-нибудь синий слайдер с овальными краями в общем стиле.

@Shvidunov
Copy link

Не понял чуток. Все как и должно быть или они черные на темной теме?
их не видно в темной теме

image

@Ledchig
Copy link
Contributor Author

Ledchig commented Jan 9, 2024

Еще кнопки съехали в хэдэре при широком экране)

@Shvidunov
Copy link

не совсем понял откуда тянется почта при регистрации через Git

image

@Ledchig
Copy link
Contributor Author

Ledchig commented Jan 9, 2024

Все правки сделал по замечаниям. Единственное от чего сейчас чуток глаз дергается - это срабатывание ховера на кнопках войти/регистрация.
Не смог воспроизвести цвета в факе на разных темах. У меня на убунту в Firefox и Chromium все как надо отображается. Может на рендере в тот момент еще старая версия была? Посмотри еще раз.
Про почту не знаю.

@Shvidunov
Copy link

потерялся лого Хекслета

image

@Ledchig
Copy link
Contributor Author

Ledchig commented Jan 14, 2024

@Shvidunov это тема была вручную изменена после загрузки сайта?

@Shvidunov
Copy link

@Shvidunov это тема была вручную изменена после загрузки сайта?

да, тема темная, переключена вручную после авторизации.

@Ledchig
Copy link
Contributor Author

Ledchig commented Jan 14, 2024

@Shvidunov Понял почему. Я тему пробиваю по бразуеру, а когда ее меняешь в кабинете и выходишь, то в бразуере остается системная тема, а выбранная тема прописывается в localStorage и я ее не вижу. В доке бутстрапа увидел как нужно.

@dzencot
Copy link
Collaborator

dzencot commented Feb 2, 2024

Ребят, тут еще идет работа?

@Ledchig
Copy link
Contributor Author

Ledchig commented Feb 2, 2024

Вроде как готово. Тестеры ни чего нового давно не писали. Если готовы принять, то мне надо бы подготовить к мерджу изменения? Просто, я все изменения в отдельной директории делал и в локалях нужно текста добавить перед мерджем.

@fey
Copy link
Contributor

fey commented Feb 4, 2024

а мы сможем выкатить лендинг на отдельной странице, чтобы посмотреть, как она выглядит и что-нибудь поправить? Например на runit.hexlet.ru/landing

@dzencot
Copy link
Collaborator

dzencot commented Feb 8, 2024

Залил новый лендинг https://runit-new-landing.onrender.com/

@Ledchig
Copy link
Contributor Author

Ledchig commented Feb 8, 2024

А, извиняюсь, на тестовые сильно отвлекся и забыл скинуть. У меня он развернут был по этой ссылке https://ledchig-runit.onrender.com/

@fey
Copy link
Contributor

fey commented Feb 8, 2024

а мы можем именно отдельную страницу сделать пока что? Чтобы на главной выводился старый лендинг, а на /landing - новый?
Просто за это время добавились всякие фичи, типа редактирование кода без реги, и надо бы ничо не сломать

@Ledchig
Copy link
Contributor Author

Ledchig commented Feb 8, 2024

Просто путь добавить?

@fey
Copy link
Contributor

fey commented Feb 8, 2024

ага

@fey
Copy link
Contributor

fey commented Feb 8, 2024

я кстати задеплоил новую версию, можете посмотреть, что добавилось

@Ledchig
Copy link
Contributor Author

Ledchig commented Feb 8, 2024

Вечером пропишу путь. Там дефолтный оставить как есть или сделать на новый лендинг?

@fey
Copy link
Contributor

fey commented Feb 8, 2024

не совсем понял про дефолтный)

Грубо говоря на главной странице у нас ничего не меняется. Просто если мы вводим в адресную строку runit.hexlet.ru/landing, то можем зайти на страницу нового лендинга. Он будет полностью рабочим и так далее, просто не показывается пользователям по дефолту.

@Ledchig
Copy link
Contributor Author

Ledchig commented Feb 8, 2024

Вот про это и был вопрос)

@Ledchig
Copy link
Contributor Author

Ledchig commented Feb 9, 2024

Сделал. Можно смотреть

@fey
Copy link
Contributor

fey commented Feb 10, 2024

@dzencot глянь одним глазом, ок ли щас по роутингу и тд и давай мержить. Скорее всего в лендос потребуются изменения, но будем делать их постепенно, а щас я не вижу смысла блочить ПР. И так много работы было сделано =)

@dzencot dzencot merged commit d8eeb63 into hexlet-rus:main Feb 13, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Сделать новый лендинг
5 participants